Skip to content

reenable acceptance test for server encoding #1547

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

reenable acceptance test for server encoding #1547

wants to merge 1 commit into from

Conversation

bastelfreak
Copy link
Collaborator

Summary

Provide a detailed description of all the changes present in this pull request.

Additional Context

Add any additional context about the problem here.

  • Root cause and the steps to reproduce. (If applicable)
  • Thought process behind the implementation.

Related Issues (if any)

Mention any related issues or pull requests.

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified. (For example puppet apply)

@bastelfreak bastelfreak self-assigned this Oct 26, 2023
@ekohl ekohl linked an issue Oct 26, 2023 that may be closed by this pull request
setting the encoding parameter causes a dependency cycle #1543
Closed
@ekohl
Copy link
Collaborator

ekohl commented Oct 26, 2023

Looks relevant and the same failure as #1543

        (Anchor[postgresql::server::service::begin::main] => Postgresql_psql[Set template1 encoding to UTF8] => Postgresql::Server::Instance::Late_initdb[main] => Postgresql::Server::Instance::Initdb[main] => Class[Postgresql::Server::Initdb] => Postgresql_conf[port_for_instance_main] => Class[Postgresql::Server::Service] => Postgresql::Server::Instance::Service[main] => Anchor[postgresql::server::service::begin::main])\nTry the '--graph' option and opening the resulting '.dot' file in OmniGraffle or GraphViz
       Error: Failed to apply catalog: One or more resource dependency cycles detected in graph

This was referenced Oct 26, 2023
Closed
Open
@smortex smortex mentioned this pull request Jan 3, 2024
Merged
@bastelfreak bastelfreak force-pushed the tests3 branch 2 times, most recently from 6f38271 to 3e6ac76 Compare April 10, 2024 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alexjfisher alexjfisher Awaiting requested review from alexjfisher alexjfisher is a code owner

@deric deric Awaiting requested review from deric deric is a code owner

@ekohl ekohl Awaiting requested review from ekohl ekohl is a code owner

@SimonHoenscheid SimonHoenscheid Awaiting requested review from SimonHoenscheid SimonHoenscheid is a code owner

@smortex smortex Awaiting requested review from smortex smortex is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@bastelfreak bastelfreak

Labels
community maintenance
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

setting the encoding parameter causes a dependency cycle
2 participants
@bastelfreak @ekohl